Skip to content

Bundle UMD with rollup#449

Merged
jquense merged 5 commits into
reactjs:masterfrom
TrySound:rollup
Feb 13, 2019
Merged

Bundle UMD with rollup#449
jquense merged 5 commits into
reactjs:masterfrom
TrySound:rollup

Conversation

@TrySound

@TrySound TrySound commented Jan 14, 2019

Copy link
Copy Markdown
Contributor
-Webpack production bundle is 18407b (no gzip)
+Rollup production bundle is 14975b (no gzip)

```diff
-Webpack production bundle is 18407b (no gzip)
+Rollup production bundle is 14975b (no gzip)
```
@TrySound

Copy link
Copy Markdown
Contributor Author

Oops, it's 14975b. Didn't expected such behaviour from babel config. I don't like custom presets for this. They need to be learned.

@jquense jquense left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm @silvenon can you also take a look

@TrySound

Copy link
Copy Markdown
Contributor Author

Guys, I already have esm support locally. Will push it after merging this PR.

@TrySound

Copy link
Copy Markdown
Contributor Author

Hi @silvenon. Could you take a look at this?

@TrySound

Copy link
Copy Markdown
Contributor Author

@silvenon Friendly ping

@TrySound

TrySound commented Feb 9, 2019

Copy link
Copy Markdown
Contributor Author

Sorry for noise. Just want to get things done :)
/cc @silvenon

@TrySound

Copy link
Copy Markdown
Contributor Author

@silvenon Ping :)

@jquense jquense merged commit ac15233 into reactjs:master Feb 13, 2019
@jquense

jquense commented Feb 13, 2019

Copy link
Copy Markdown
Contributor

I think we can move this along, happy to any feedback in retro

@TrySound TrySound deleted the rollup branch February 13, 2019 20:47
errendir added a commit to errendir/react-transition-group that referenced this pull request Feb 14, 2019
…d-compatible

* origin/master:
  chore: bundle UMD with rollup (reactjs#449)
  chore(release): 2.5.3 [skip ci]
  fix: strip custom prop-types in production (reactjs#448)
  chore: fix storybook (reactjs#445)
  chore(release): 2.5.2 [skip ci]
  fix: pass appear to CSSTransition callbacks (reactjs#441)
  docs: Fix type declaration comment (reactjs#439)
  chore(release): 2.5.1 [skip ci]
  fix: prevent calling setState in TransitionGroup if it has been unmounted (reactjs#435)
errendir added a commit to errendir/react-transition-group that referenced this pull request Feb 14, 2019
…d-compatible

* origin/master:
  chore: bundle UMD with rollup (reactjs#449)
  chore(release): 2.5.3 [skip ci]
  fix: strip custom prop-types in production (reactjs#448)
  chore: fix storybook (reactjs#445)
  chore(release): 2.5.2 [skip ci]
  fix: pass appear to CSSTransition callbacks (reactjs#441)
  docs: Fix type declaration comment (reactjs#439)
  chore(release): 2.5.1 [skip ci]
  fix: prevent calling setState in TransitionGroup if it has been unmounted (reactjs#435)
@silvenon

silvenon commented Feb 14, 2019

Copy link
Copy Markdown
Collaborator

So sorry guys. Thanks a lot @TrySound for your work and thanks @jquense for reviewing ❤️

@jquense

jquense commented Feb 26, 2019

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants